fix(create-frontend): pin jQuery to 3.7.1 for basercms4 projects#901
fix(create-frontend): pin jQuery to 3.7.1 for basercms4 projects#901teppei-d wants to merge 2 commits into
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
teppei-d
requested review from
YusukeHirao,
arktds,
dz-furukawa and
yusasa16
as code owners
| delete pkg.devDependencies['@burger-editor/local']; | ||
| pkg.dependencies['@burger-editor/css'] = '2'; | ||
| pkg.dependencies['jquery'] = 'latest'; | ||
| pkg.dependencies['jquery'] = '3.7.1'; // jQuery 4系で削除されたAPIとの互換性のため3系の最新版を利用 |
There was a problem hiding this comment.
3.7.1 に固定しなければいけない理由をもう少し具体的にコメントで残したほうがいいかなと思いました！
互換性がないコードは isFunction() だけでしょうか？
それらの互換性がない関数はどのファイルが実行していますか？colorbox？だけ？
There was a problem hiding this comment.
@arktds
はい。デモページでエラーになっていたのはisFunction()だけでした。 原因は bge_functions.min.js → jquery.colorbox-min.js の流れで、colorbox が jQuery 4.0 で削除された$.isFunction()に依存しているためです。
以下のようにコメントを修正予定ですがいかがでしょうか？
// colorbox が jQuery 4.0 で削除された $.isFunction() に依存するため 3.x 最新版（3.7.1）に固定
There was a problem hiding this comment.
@teppei-d
colorbox は isFunction() 以外にも互換性がなくなった関数を使用しているようです。
今回はたまたま isFunction() だけ使用していたか、isFunction() のエラーを回避しても次に別のエラーが出る可能性もありそうです。
したがって、やはり isFunction() という関数名は明記せず、「colorbox」というパッケージ名だけは明記したほうが良さそうです！
例えば、次のようなニュアンスだと良いかなと思います！
// colorbox が jQuery 4系と互換性がないため、3系の最新版に固定
There was a problem hiding this comment.
@arktds
ご指摘ありがとうございます！「colorbox」というパッケージ名を明記するコメントに変更しております。
お手隙の際に、ご確認よろしくお願いいたします！
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
4 participants
背景
plopfile.jsでjquery: 'latest'と指定していたため、2026年1月リリースの jQuery 4.0.0 が取得されていた。colorbox が jQuery 4系と互換性がないため、スライドショーなどが動作しない状態になっていた（jQuery 4.0 Upgrade Guide）。
変更内容
pkg.dependencies['jquery'] = 'latest'→'3.7.1'に固定（jQuery 3系最新版）jqueryが3.7.1であることを検証するテストを追加動作確認
jQuery 3.7.1 を適用した環境で、動作しなくなっていたスライドショーが正常に動作することを確認済み。
🤖 Generated with Claude Code